Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

03-Implemented changes for custom python test #167

Merged
merged 7 commits into from
Nov 14, 2024

Conversation

rquidute
Copy link
Contributor

What changed

Updated the files related to custom python tests in order to the be in accordance with the changes with the way to retrieve test steps names.

Related Issue

project-chip/certification-tool#484

Testing

Unit test updated and passing
Screenshot 2024-11-12 at 20 21 53

TH UI List the custom test case
Screenshot 2024-11-12 at 20 22 27

TH UI execution runs successfully the Custom Python test case.
Screenshot 2024-11-12 at 20 22 50

Copy link
Contributor

@hiltonlima hiltonlima left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just check the CI errors

app/tests/conftest.py Outdated Show resolved Hide resolved
@rquidute
Copy link
Contributor Author

spelling and unit test are still failing. At first, it does not make sense to me. I will merge this PR now and check this afterwards. In case, I will open a new PR to fix this CI issues.

@rquidute rquidute merged commit 403ad1f into spring2025 Nov 14, 2024
6 of 8 checks passed
@rquidute rquidute changed the title Implemented changes for custom python test 03-Implemented changes for custom python test Nov 18, 2024
hiltonlima pushed a commit that referenced this pull request Dec 17, 2024
* Implemented changes for custom python test

* code review

* Code review

* Code reviewd

* Fixed code violations

* Chanegs after code review

* Remove unused import
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants